Skip to content

[VALIDATED] [REFACTOR] Standardize LLM Provider Configurations#14

Merged
HOLYKEYZ merged 3 commits intomasterfrom
bot/refactor-llm-providers-1773700640
Mar 17, 2026
Merged

[VALIDATED] [REFACTOR] Standardize LLM Provider Configurations#14
HOLYKEYZ merged 3 commits intomasterfrom
bot/refactor-llm-providers-1773700640

Conversation

@joe-gemini-bot
Copy link
Contributor

Hey @HOLYKEYZ! Joseph, I've found an improvement for you.

Problem / Gap

The current configuration in backend/app/core/config.py contains duplicated COUNCIL_ENABLE_PARALLEL settings and ambiguous entries for GEMINI2_API_KEY, GEMINI2_MODEL, GEMINI2_TIMEOUT, GROK2_API_KEY, GROK2_MODEL, and GROK2_TIMEOUT. The README mentions 'Gemini 2' as a model, which is already covered by GEMINI_MODEL: 'gemini-2.5-flash', making the GEMINI2 entries redundant as separate provider configurations. Similarly, the GROQ_API_KEY is aliased with GROK_API_KEY, but then separate GROK2 entries exist, creating confusion about whether 'Grok' is a distinct provider or a misnomer for 'Groq'. The corresponding 'Grok' entry in extension/manifest.json should also be removed if 'Grok' is not a distinct, supported platform.

Solution & Insight

To address the inconsistency and redundancy in LLM provider configurations, we will remove the redundant COUNCIL_ENABLE_PARALLEL setting and the ambiguous GEMINI2 and GROK2 entries from backend/app/core/config.py. We will also remove the 'Grok' host permission from extension/manifest.json.

Impact

This change improves code clarity, reduces potential for misconfiguration, and ensures consistency with the documented LLM Council members.


Validated by Triple-AI: Scanner (Gemini 2.5 Flash) → Executor (Llama 3.3 70B) → Reviewer (Gemini 2.5 Flash)

Generated autonomously by Mayo 🤖

@vercel
Copy link

vercel bot commented Mar 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
intellect-safe Ready Ready Preview, Comment Mar 17, 2026 2:19pm

@HOLYKEYZ HOLYKEYZ merged commit c56aaf7 into master Mar 17, 2026
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant